-
-
Notifications
You must be signed in to change notification settings - Fork 8.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(ci): run V1 Build and Migration CLI tests conditionally on CI #3863
chore(ci): run V1 Build and Migration CLI tests conditionally on CI #3863
Conversation
Deploy preview for docusaurus-2 ready! Built without sensitive environment variables with commit d0d667b |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-3863--docusaurus-2.netlify.app/classic/ |
Size Change: 0 B Total Size: 157 kB ℹ️ View Unchanged
|
Thanks :) |
outputs: | ||
migration: ${{ steps.filter.outputs.migration }} | ||
steps: | ||
- uses: dorny/paths-filter@v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
strange that it works because on another brranch it fails due to missing git checkout step (I think)
Motivation
This PR is an attempt to optimize CI workflows a bit more to improve the time required to check every PR.
I have utilized the
@dorny/paths-filter
action to add check based on the modified files in the PR so now:packages/docusuaurs-1.x
orpackages/docusuaurs-init-1.x
files have been changed.packages/docusuaurs-migration
includes the changes.This is the first time I have used this action, so the initial version might be not correct. Any tips and ideas for further optimizations are welcome!
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Run the CI! 😉
Related PRs